-
Notifications
You must be signed in to change notification settings - Fork 423
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UCT/UD: Fix issue N544 (CREQ assert) #953
Conversation
Fixed: - If CREQ and CREP to peer are lost, peer may stay unconnected - type == UCT_UD_PACKET_CREQ assertion
Test PASSed. |
Test FAILed. |
@alex-mikheev, could you please look thru? |
Test FAILed. |
@@ -578,6 +579,9 @@ void uct_ud_ep_process_rx(uct_ud_iface_t *iface, uct_ud_neth_t *neth, unsigned b | |||
goto out; | |||
} | |||
if (neth->packet_type & UCT_UD_PACKET_FLAG_CTL) { | |||
/* No need to track duplications, CREP always goes | |||
* with ACK_REQ flag */ | |||
ep->rx.ooo_pkts.head_sn = neth->psn; | |||
uct_ud_ep_rx_ctl(iface, ep, (uct_ud_ctl_hdr_t *)(neth + 1)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it think it is better to add this line in uct_ud_ep_rx_ctl()
Test FAILed. |
Test PASSed. |
Test PASSed. |
👍 |
Fix issue introduced by openucx#953 - when getting duplicate CREP with lower psn, the current expected psn should not be affected. Otherwise packets already acknowledged would be considered missing.
Fix issue introduced by openucx#953 - when getting duplicate CREP with lower psn, the current expected psn should not be affected. Otherwise packets already acknowledged would be considered missing.
Fixed:
fixes #544